Migration tenders-finder to Tinyfish SDK #175
Migration tenders-finder to Tinyfish SDK #175KrishnaAgarwal7531 wants to merge 3 commits intotinyfish-io:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
simantak-dabhade
left a comment
There was a problem hiding this comment.
Thanks for the migration @KrishnaAgarwal7531 — the SDK usage in scrape/route.ts looks clean (client.agent.stream, proper event handling, no raw fetch). Security check passed — no leaked keys this time.
Build is broken — needs fixes before merge.
1. Delete 38 unused shadcn/ui component files (blocker)
npm run build fails because files like accordion.tsx import @radix-ui/react-accordion which isn't installed. There are 38 unused component files under src/components/ui/ that the app doesn't use. Please delete all of them — the app only needs these:
button.tsx, dialog.tsx, input.tsx, label.tsx, separator.tsx, sheet.tsx, skeleton.tsx, toast.tsx, toggle.tsx, tooltip.tsx
Delete everything else in src/components/ui/.
2. Remove unused groq-sdk dependency
groq-sdk is in package.json but never imported anywhere in src/. Please:
- Remove
"groq-sdk"frompackage.json - Remove the
GROQ_API_KEYlines from.env.example - Remove the Groq section from
README.mdenvironment variables
(Also: Groq is no longer accepting new signups, so even if it were used we'd need to switch it.)
3. Bump Next.js to 16.x
next@15.3.6 has a known security vulnerability. Please bump to "next": "^16.2.4" (matches the rest of the cookbook).
What looks good ✓
- SDK usage is correct:
client.agent.stream()inside the request handler (not module-level) - No Mino references, no hardcoded secrets
.env.examplecorrectly named with canonical sign-up URL.gitignoreproperly covers.envvariants- Supabase code fully removed
- Old Vite artifacts cleaned up
Once the build passes (npm install && npm run build), this is good to merge. Thanks!
Migrates Tenders Finder to the official TinyFish SDK. /api/scrape uses client.agent.stream with typed EventType.STARTED, EventType.STREAMING_URL, EventType.PROGRESS, and EventType.COMPLETE + RunStatus.COMPLETED. No Mino refs, no Supabase, .env.example present.
Added .gitignore and README.md.